Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[master] Update dependencies from dotnet/arcade #574

Merged

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jul 28, 2020

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

  • Subscription: 66bf0ed9-59cb-4fe0-6937-08d6a0d7585c
  • Build: 20200724.1
  • Date Produced: 7/24/2020 3:41 PM
  • Commit: f6192d1e284a08ac05041d05fa6e60dec74b24f5
  • Branch: refs/heads/master
  • Updates:
    • Microsoft.DotNet.Arcade.Sdk: from 5.0.0-beta.20117.3 to 5.0.0-beta.20374.1

…724.1

Microsoft.DotNet.Arcade.Sdk
 From Version 5.0.0-beta.20117.3 -> To Version 5.0.0-beta.20374.1
@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Jul 28, 2020

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • All Checks Successful Unsuccessful checks: public-CI, public-CI (Windows SDK Release Test), public-CI (Windows SDK Debug Test), public-CI (Windows Samples Release Test), public-CI (Windows Samples Debug Test)

@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner July 29, 2020 00:58
@JoeRobich JoeRobich force-pushed the darc-master-5ea68984-63a6-4436-99ec-565f4648f17b branch from 8de48b1 to aabdb0e Compare July 29, 2020 01:03
@JoeRobich
Copy link
Member

JoeRobich commented Jul 29, 2020

If I don't define DOTNET_ROOT, all the .netcoreapp3.1 MSTest and NUnit tests fail with (https://dev.azure.com/dnceng/public/_build/results?buildId=749215&view=results)

Microsoft (R) Test Execution Command Line Tool Version 16.7.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.
! Testhost process exited with error: A fatal error occurred, the required library hostfxr.dll could not be found.
If this is a self-contained application, that library should exist in [F:\workspace\_work\1\s\artifacts\bin\Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests\Release\netcoreapp3.1\].
If this is a framework-dependent application, install the runtime in the default location [C:\Program Files\dotnet] or use the DOTNET_ROOT environment variable to specify the runtime location.
. Please check the diagnostic logs for more information.

Test Run Aborted.
Results File: F:\workspace\_work\1\s\artifacts\TestResults\Release\Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests_netcoreapp3.1_x64.xml
=== COMMAND LINE ===
F:\workspace\_work\1\s\.dotnet\dotnet.exe test F:\workspace\_work\1\s\artifacts\bin\Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests\Release\netcoreapp3.1\Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests.dll --logger:"console;verbosity=normal" --logger:"trx;LogFileName=Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests_netcoreapp3.1_x64.xml" "--ResultsDirectory:F:\workspace\_work\1\s\artifacts\TestResults\Release" "--Framework:.NETCoreApp,Version=v3.1" > "F:\workspace\_work\1\s\artifacts\log\Release\Microsoft.CodeAnalysis.VisualBasic.CodeRefactoring.Testing.MSTest.UnitTests_netcoreapp3.1_x64.log" 2>&1

If I do define DOTNET_ROOT, all the .netcoreapp1.1 MSTest and NUnit tests fail with (https://dev.azure.com/dnceng/public/_build/results?buildId=749296&view=results)

Microsoft (R) Test Execution Command Line Tool Version 16.7.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.
! Unable to find F:\workspace\_work\1\s\artifacts\bin\Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests\Release\netcoreapp1.1\testhost.dll. Please publish your test project and retry.

Test Run Aborted.
Results File: F:\workspace\_work\1\s\artifacts\TestResults\Release\Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests_netcoreapp1.1_x64.xml
=== COMMAND LINE ===
F:\workspace\_work\1\s\.dotnet\dotnet.exe test F:\workspace\_work\1\s\artifacts\bin\Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests\Release\netcoreapp1.1\Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests.dll --logger:"console;verbosity=normal" --logger:"trx;LogFileName=Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests_netcoreapp1.1_x64.xml" "--ResultsDirectory:F:\workspace\_work\1\s\artifacts\TestResults\Release" "--Framework:.NETCoreApp,Version=v1.1" > "F:\workspace\_work\1\s\artifacts\log\Release\Microsoft.CodeAnalysis.Testing.Verifiers.MSTest.UnitTests_netcoreapp1.1_x64.log" 2>&1

@dotnet-maestro
Copy link
Contributor Author

Auto-Merge Status

This pull request has not been merged because Maestro++ is waiting on the following merge policies.

  • All Checks Successful Unsuccessful checks: public-CI, public-CI (Windows SDK Release Test), public-CI (Windows SDK Debug Test), public-CI (Windows Samples Release Test), public-CI (Windows Samples Debug Test)

@JoeRobich
Copy link
Member

JoeRobich commented Jul 29, 2020

Also, we are seeing this implicit package reference error This has been resolved.

F:\workspace\_work\1\s\.dotnet\sdk\5.0.100-preview.6.20310.4\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.DefaultItems.targets(257,5): error NETSDK1023: A PackageReference for 'xunit' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs [F:\workspace\_work\1\s\samples\Shared\UnitTestFramework\Roslyn.UnitTestFramework.csproj]

If we remove the package reference, we see this error in the following build

CodeActionProviderTestFixture.cs(10,7): error CS0246: The type or namespace name 'Xunit' could not be found (are you missing a using directive or an assembly reference?) [F:\workspace\_work\1\s\samples\Shared\UnitTestFramework\Roslyn.UnitTestFramework.csproj]

@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Jul 29, 2020

Auto-Merge Status

This pull request has been merged because the following merge policies have succeeded.

  • ✔️ All Checks Successful Succeeded - Successful checks: public-CI, public-CI (Windows SDK Sign), public-CI (Windows SDK Pack), public-CI (Windows SDK Release Test), public-CI (Windows SDK Debug Test), public-CI (Windows Samples Release Test), public-CI (Windows Samples Debug Test)

@JoeRobich JoeRobich force-pushed the darc-master-5ea68984-63a6-4436-99ec-565f4648f17b branch from ef6a384 to c82221f Compare July 29, 2020 05:53
dotnet-maestro bot and others added 3 commits July 29, 2020 16:02
…724.1

Microsoft.DotNet.Arcade.Sdk
 From Version 5.0.0-beta.20117.3 -> To Version 5.0.0-beta.20374.1
Support for netcoreapp1.1 was removed in microsoft/vstest#2067 and
microsoft/vstest#2192, so we can no longer take updates to this package.
@dotnet-maestro dotnet-maestro bot merged commit e2e9d63 into master Jul 29, 2020
@dotnet-maestro dotnet-maestro bot deleted the darc-master-5ea68984-63a6-4436-99ec-565f4648f17b branch July 29, 2020 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants